Skip to content

Conversation

@edthri
Copy link
Contributor

@edthri edthri commented Jan 28, 2026

I couldn't really make sense of Settings -> Extensions as it doesn't correlate with what i'm seeing in OIE.

Copy link
Member

@tonygermano tonygermano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, thanks for contributing! Your edits look good. I want you to be aware that we just merged in a LICENSE in #9 that will apply to your contribution if merged as well as an expanded README. See the section on the DCO sign-off. I'll assume that if you sign-off on your commit after this comment that you are also agreeing to the license terms. Let me know if you need assistance working with git.

@edthri
Copy link
Contributor Author

edthri commented Jan 30, 2026

I'm not mega familiar with git/github. Is it easier to close this PR? I can update my fork and make the same edits and then submit a new PR including DCO signoff in the commit?

I couldn't really make sense of Settings -> Extensions as it doesn't correlate with what i'm seeing in OIE.

Signed-off-by: edthri <riordaned@gmail.com>
Signed-off-by: Tony Germano <tony@germano.name>
@tonygermano
Copy link
Member

I'm not mega familiar with git/github. Is it easier to close this PR? I can update my fork and make the same edits and then submit a new PR including DCO signoff in the commit?

@edthri I've updated the commit to add the signoff for you. If you just want to respond to this comment with your final approval that everything looks good and you agree to submit your changes under the new license terms I'll get it merged.

@edthri
Copy link
Contributor Author

edthri commented Feb 5, 2026

I agree to submit my changes under the new license terms. Approval given. Thanks,

@tonygermano tonygermano merged commit f0f8788 into OpenIntegrationEngine:main Feb 5, 2026
1 check passed
@edthri edthri deleted the edthri-patch-1 branch February 5, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants